-
Notifications
You must be signed in to change notification settings - Fork 6
EdgeDB - Partner and Organization Queries #3105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
fc2836a
to
d77b578
Compare
0674218
to
bc274ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Want to rebase this and continue here @brentkulwicki?
The yarn changes should be reverted.
There should be other examples of edgedb repos you can follow now.
Any notes on here? Maybe this will be fine now that I am on a new machine, but I felt like this one was done (outside of all the errors I had that I thought were the issues locally) |
f6863a1
to
503de3e
Compare
503de3e
to
5afced1
Compare
1b8c711
to
7f17053
Compare
🗞 GraphQL SummaryView schema changes@@ -3240,9 +3240,9 @@
}
type Organization implements Resource {
acronym: SecuredStringNullable!
- address: SecuredString!
+ address: SecuredStringNullable!
avatarLetters: String
"""Whether the requesting user can delete this resource"""
canDelete: Boolean!
@@ -3412,9 +3412,9 @@
}
type Partner implements Pinnable & Postable & Resource {
active: SecuredBoolean!
- address: SecuredString!
+ address: SecuredStringNullable!
"""Whether the requesting user can delete this resource"""
canDelete: Boolean!
countries: SecuredLocations!
@@ -3434,9 +3434,9 @@
organization: SecuredOrganization!
"""Does the requesting user have this pinned?"""
pinned: Boolean!
- pmcEntityCode: SecuredString!
+ pmcEntityCode: SecuredStringNullable!
pointOfContact: SecuredUser!
"""List of posts belonging to the parent node."""
posts(input: PostListInput = {count: 25, order: DESC, page: 1, sort: "createdAt"}): SecuredPostList!
🚨 Breaking Changes
|
Still some fixes to make here 🤔 |
922f2ba
to
ada4443
Compare
Co-authored-by: Carson Full <carson_full@tsco.org>
Co-authored-by: Carson Full <carson_full@tsco.org>
Co-authored-by: Carson Full <carson_full@tsco.org>
Co-authored-by: Carson Full <carson_full@tsco.org>
4380483
to
51c93e6
Compare
Monday task
Reason for this PR
Migrate Partner and Organization Queries to edgedb
Ready for review checklist